Skip to content

refactor(ci): port source and test report checks to TypeScript - #768

Open
kmbroai wants to merge 3 commits into
mainfrom
dev/kyleb/remove-python-ci-utilities
Open

refactor(ci): port source and test report checks to TypeScript#768
kmbroai wants to merge 3 commits into
mainfrom
dev/kyleb/remove-python-ci-utilities

Conversation

@kmbroai

@kmbroai kmbroai commented Aug 31, 2026

Copy link
Copy Markdown
Collaborator

Summary

Replace the portable-source checker and JUnit inventory comparator with typed TypeScript utilities. Compile them with the existing tsc and run the emitted JavaScript on Node 22.13.0, removing these utilities' Python dependency without relying on experimental type stripping.

Changes

  • Replace two Python utilities and the source-checker test with three typed .mts files.
  • Preserve source byte limits, Markdown checks, tracked-file handling, JUnit outcomes, duplicate detection, and shard matching.
  • Add build:ci to emit adjacent .mjs files; generated files are ignored and excluded from the published SDK.
  • Compile before running the utilities in CI, including reduced Markdown checks; update help, contributor commands, and regression tests.
  • Include the sources in strict TypeScript and formatting checks. Use development dependencies for XML parsing and glob matching.

Testing

  • Node 22.13.0: TypeScript compilation, canonical plugin source check, and all 9 emitted checker tests passed without experimental flags.
  • Compared 26 source/XML cases against the original Python implementations: matching results.
  • SDK type and formatting checks passed.
  • Full SDK suites with seeds 12345 and 2474797289: each had 2,137 passes, 41 skips, and zero failures.
  • Independent code review and the final committed-head review gate passed with no actionable findings.

Risk and rollout

The affected CI jobs install SDK development dependencies and compile the utilities before running them. Test fixtures compile isolated copies so parallel runs do not share generated files.

Public disclosure review

  • No customer, partner, prospect, or user identities, data, or identifying details are included.
  • No credentials, personal data, private source, scan findings, or nonpublic links or tickets are included.
  • I reviewed the branch name, title, description, commits, changes, comments, logs, screenshots, attachments, and links for public disclosure.

@kmbroai
kmbroai marked this pull request as draft August 31, 2026 23:14
@kmbroai kmbroai changed the title refactor(ci): port source and test report checks to JavaScript refactor(ci): port source and test report checks to TypeScript Aug 31, 2026
@kmbroai
kmbroai marked this pull request as ready for review August 31, 2026 23:26
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Aug 31, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
🔒 Security Review Completed 2026-09-02T17:52:07.558027Z 8ac81f7 New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant